[DEV-228/FE] feat: 내 면접 모아보기 질문탭 mock-api 연결#379
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @HIHJH, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 '내 면접 모아보기' 페이지의 질문 탭 기능을 전면적으로 개선합니다. 기존의 모의 데이터를 사용하던 질문 목록을 실제 API와 연동하고, 자주 받은 질문 섹션과 전체 질문 검색 및 필터링 기능을 도입하여 사용자가 자신의 면접 질문 데이터를 효율적으로 관리하고 탐색할 수 있도록 합니다. 특히, 검색 결과에 무한 스크롤을 적용하여 대량의 데이터를 부드럽게 탐색할 수 있게 하였으며, 질문 필터링을 통해 원하는 질문을 쉽게 찾을 수 있도록 기능성을 강화했습니다. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
이번 PR은 '내 면접 모아보기'의 질문 탭을 mock API와 성공적으로 연결하고, 더 나은 구조와 상태 관리를 위해 컴포넌트를 리팩토링했습니다. 페이지네이션을 위한 useInfiniteQuery와 무한 스크롤을 위한 IntersectionObserver의 사용이 돋보입니다. 커스텀 훅과 매퍼 함수를 통해 코드도 잘 정리되었습니다. 코드 품질을 더욱 향상시키기 위해 React key 사용, 타입 안전성, 타입 정의 간소화에 대한 몇 가지 제안 사항을 드립니다.
frontend/src/features/dashboard/my-interviews/components/questions/mappers.ts
Show resolved
Hide resolved
...nd/src/features/dashboard/my-interviews/components/questions/list/useInfiniteQuestionList.ts
Outdated
Show resolved
Hide resolved
frontend/src/features/dashboard/my-interviews/components/questions/list/QuestionListSection.tsx
Show resolved
Hide resolved
forhyundaisofteer
left a comment
There was a problem hiding this comment.
고생하셨습니다!! 남은거 다 머지 되고 나서 mock 빼봐요!
관련 이슈
close #323
작업한 내용
PR 리뷰시 참고할 사항
frontend/src/features/dashboard/my-interviews/components/questions폴더 위주로 봐주시면 됩니다!현재 swagger에는 request param 형태가 pageable 객체로 지정되어 있는데, 실제로는 page, size, sort 각각 param을 넣어주어야 해서 BE분들께 수정 요청드릴 예정입니다! 그 전까지는 임시로 api schema 파일을 수정하여 작업해두었습니다. 5620609
참고 자료 (링크, 사진, 예시 코드 등)